Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

doc: remove /retest all command for Jenkins jobs #3957

Merged
merged 1 commit into from
Jul 4, 2023
Merged

doc: remove /retest all command for Jenkins jobs #3957

merged 1 commit into from
Jul 4, 2023

Conversation

nixpanic
Copy link
Member

@nixpanic nixpanic commented Jun 29, 2023 •
edited
Loading

/retest all causes a spike in resource consumption in Jenkins and the OpenShift cluster kills the Pod. That means tests are not fully running yet, and results never arrive back in the PR. Instead of /retest all, the ok-to-test label can be used to trigger required tests with a slight delay between each command.

Depends-on: #3965

Show available bot commands

These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

  • /retest ci/centos/<job-name>: retest the <job-name> after unrelated
    failure (please report the failure too!)
  • /retest all: run this in case the CentOS CI failed to start/report any test
    progress or results

@mergify mergify bot added ci/skip/e2e skip running e2e CI jobs ci/skip/multi-arch-build skip building on multiple architectures component/docs Issues and PRs related to documentation labels Jun 29, 2023
@Madhu-1 Madhu-1 requested a review from a team June 29, 2023 11:09
@nixpanic
Copy link
Member Author

nixpanic commented Jul 3, 2023

@Mergifyio queue

@mergify
Copy link
Contributor

mergify bot commented Jul 3, 2023 •
edited
Loading

queue

🛑 The pull request has been removed from the queue default

The queue conditions cannot be satisfied due to failing checks.

You can take a look at Queue: Embarked in merge train check runs for more details.

In case of a failure due to a flaky test, you should first retrigger the CI.
Then, re-embark the pull request into the merge queue by posting the comment
@mergifyio refresh on the pull request.

@nixpanic nixpanic force-pushed the doc/jobs/remove-retest-all branch from d397025 to ba733b1 Compare July 3, 2023 15:39
@mergify mergify bot added the ok-to-test Label to trigger E2E tests label Jul 3, 2023
@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.25

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.26

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.27

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.25

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.26

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.27

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.25

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.26

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.27

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Jul 3, 2023
@nixpanic
Copy link
Member Author

nixpanic commented Jul 3, 2023

@Mergifyio queue

Oh, I thought this was a PR for the ci/centos branch...

@nixpanic
Copy link
Member Author

nixpanic commented Jul 4, 2023

@Mergifyio rebase

@mergify
Copy link
Contributor

mergify bot commented Jul 4, 2023

rebase

✅ Nothing to do for rebase action

@nixpanic
Copy link
Member Author

nixpanic commented Jul 4, 2023

Needs rebasing once #3965 is merged.

@nixpanic
Copy link
Member Author

nixpanic commented Jul 4, 2023

@Mergifyio rebase

@nixpanic
doc: remove /retest all command for Jenkins jobs
755b607 
`/retest all` causes a spike in resource consumption in Jenkins and the
OpenShift cluster kills the Pod. That means tests are not fully running
yet, and results never arrive back in the PR. Instead of `/retest all`,
the `ok-to-test` label can be used to trigger required tests with a
slight delay between each command.

Signed-off-by: Niels de Vos <ndevos@ibm.com>
@mergify
Copy link
Contributor

mergify bot commented Jul 4, 2023

rebase

✅ Branch has been successfully rebased

@nixpanic nixpanic force-pushed the doc/jobs/remove-retest-all branch from ba733b1 to 755b607 Compare July 4, 2023 11:29
@nixpanic
Copy link
Member Author

nixpanic commented Jul 4, 2023

@Mergifyio queue

@mergify
Copy link
Contributor

mergify bot commented Jul 4, 2023 •
edited
Loading

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 489d0ff

@mergify mergify bot added the ok-to-test Label to trigger E2E tests label Jul 4, 2023
@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.25

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.26

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.27

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.25

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.26

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.27

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.25

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.26

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.27

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Jul 4, 2023
@mergify mergify bot merged commit 489d0ff into ceph:devel Jul 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@riya-singhal31 riya-singhal31 riya-singhal31 approved these changes

@Madhu-1 Madhu-1 Madhu-1 approved these changes

Assignees
No one assigned
Labels
ci/skip/e2e skip running e2e CI jobs ci/skip/multi-arch-build skip building on multiple architectures component/docs Issues and PRs related to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@nixpanic @ceph-csi-bot @Madhu-1 @riya-singhal31